Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update makefile deploy-rinkeby #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

wk0
Copy link

@wk0 wk0 commented Aug 5, 2022

In .env.example, variable for etherscan api key is ETHERSCAN_KEY, while in the makefile it is ETHERSCAN_API_KEY.
Adjust makefile to use ETHERSCAN_KEY so script can run using the key specified in the .env file

In .env.example, variable for etherscan api key is ETHERSCAN_KEY, while in the makefile it is ETHERSCAN_API_KEY.
Adjust makefile to use ETHERSCAN_KEY so script can run using the key specified in the .env file
@wk0
Copy link
Author

wk0 commented Aug 5, 2022

can alternatively change the .env file to use ETHERSCAN_KEY

@wk0
Copy link
Author

wk0 commented Aug 6, 2022

Hmm, still had issue using the deploy rinkeby make script, but found this issue.

foundry-rs/foundry#2435

@wk0
Copy link
Author

wk0 commented Aug 6, 2022

but then you have to switch to using forge create rather than the deploy script, so disregard dc0d2f8

@giovannidisiena
Copy link

@wk0 is there any chance you are compiling via the new intermediate representation pipeline? Verification of Yul IR bytecode on Etherscan is not currently supported, so that can also cause you to run into issues.

@PatrickAlphaC
Copy link
Contributor

It looks like you took out the broadcast option, was that intentional?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants